Skip to content

Conversation

@rvedotrc
Copy link
Contributor

@rvedotrc rvedotrc commented Apr 15, 2025

The idea of course being that, once this PR is merged, then Lint should be marked as a required check.

@rvedotrc rvedotrc marked this pull request as ready for review April 15, 2025 17:01
@adamblanchard adamblanchard self-requested a review April 20, 2025 06:52
Copy link
Contributor

@adamblanchard adamblanchard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looooove this idea. Inconsistent formatting plagues the other modules - another "easy" benefit we get from the monorepo approach is running linting across all of the content. Nice!

@rvedotrc rvedotrc requested a review from a team April 20, 2025 11:39
@rvedotrc
Copy link
Contributor Author

Looooove this idea. Inconsistent formatting plagues the other modules - another "easy" benefit we get from the monorepo approach is running linting across all of the content. Nice!

plus any future checks, such as checking for dead external links, inconsistent filename casing, missing cross-linking, etc.

Copy link

@beverlylytle beverlylytle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@adamblanchard adamblanchard merged commit 6f73903 into main Apr 21, 2025
1 check passed
@rvedotrc rvedotrc deleted the linting branch April 21, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants